Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Testing and drift testing #65

Merged
merged 6 commits into from
Dec 19, 2023
Merged

Testing and drift testing #65

merged 6 commits into from
Dec 19, 2023

Conversation

signorecello
Copy link
Collaborator

Description

This PR introduces some drift testing for noir starter.

Problem*

The task of understanding whether noir-starter was still relevant was very much dependent on people telling us that. Which means at least one person will potentially already be blocked by it.

We want to reduce that to a minimum, so this PR will run every night and attempt to build master with the latest stable. If it fails, we can configure alerts and know.

The script is also grabbing all the prereleases. It would be cool to have these tests run on every prerelease so we know in advance when the next stable will break. But that doesn't necessarily help us, we can't update the boilerplate in advance otherwise master will break 😄 Could still be useful to know it will break soon, so we can already start working on a branch

Copy link

netlify bot commented Dec 19, 2023

Deploy Preview for noir-vite-hardhat canceled.

Name Link
🔨 Latest commit 1aff40d
🔍 Latest deploy log https://app.netlify.com/sites/noir-vite-hardhat/deploys/6581db85e25a9e0008f61adf

Copy link
Collaborator

@critesjosh critesjosh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's try it and see how it goes! 🤞

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cc @catmcgee as this may be a useful reference for seeing how to get the latest release info from an external repo.

@signorecello signorecello merged commit e5b966a into main Dec 19, 2023
5 checks passed
@signorecello signorecello deleted the zpedro/ci branch December 19, 2023 19:15
@signorecello signorecello mentioned this pull request Dec 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants